Skip to content

Попова Елизавета. Отчеты. Поразрядная сортировка для вещественных чисел (тип double) с простым слиянием. Вариант 19#1350

Open
lisssq wants to merge 10 commits into
learning-process:masterfrom
lisssq:popova_e_radix_sort_for_double_REPORT
Open

Conversation

@lisssq
Copy link
Copy Markdown
Contributor

@lisssq lisssq commented May 20, 2026

Описание

  • Задача: Поразрядная сортировка для вещественных чисел (тип double) с простым слиянием
  • Вариант: 19
  • Технология: Reports

Чек-лист

  • Корневой report.md существует и читается как самостоятельный документ.
  • Заполнены seq/report.md, omp/report.md, tbb/report.md, stl/report.md, all/report.md.
  • Во всех отчётах один язык и единая терминология.
  • Во всех таблицах одинаково определены time, speedup, efficiency, workers.
  • В seq честно описан baseline, а не «почти параллельная» версия.
  • В omp расписаны shared/private/reduction/schedule.
  • В tbb объяснены blocked_range, grainsize, partitioner и способ контроля конкуренции.
  • В stl явно показано, что join вызывается после запуска всех потоков, если требуется реальный параллелизм.
  • В all указана конфигурация ranks × threads и смысл MPI-синхронизации.
  • Есть команды сборки и запуска, достаточные для воспроизведения.
  • Функциональные тесты и тесты производительности реально запускались локально.
  • Если использовались графики, указаны относительные пути и подписи.
  • Текст не содержит неподтверждённых фраз вроде «реализация оптимальна» без таблицы или профиля.
  • PR checklist выполнен: CI зелёный, clang-format/clang-tidy/тесты пройдены, report.md добавлен.

@lisssq lisssq requested a review from allnes as a code owner May 20, 2026 01:02
@lisssq
Copy link
Copy Markdown
Contributor Author

lisssq commented May 20, 2026

Данный пулл-реквест содержит:

  • реализацию технологии ALL, которая в данный момент проходит CI в общем репозитории и пока не была смержена в мастер
  • комплект отчетов (5 технологий + корневой отчет)

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented May 21, 2026

Codecov Report

❌ Patch coverage is 95.37037% with 5 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.52%. Comparing base (29821f1) to head (24a710e).
⚠️ Report is 310 commits behind head on master.

Files with missing lines Patch % Lines
...t_for_double_with_simple_merge/all/src/ops_all.cpp 95.37% 1 Missing and 4 partials ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##           master    #1350      +/-   ##
==========================================
- Coverage   83.69%   83.52%   -0.17%     
==========================================
  Files         669      817     +148     
  Lines       34251    45846   +11595     
  Branches    14301    19170    +4869     
==========================================
+ Hits        28665    38294    +9629     
- Misses       3807     5083    +1276     
- Partials     1779     2469     +690

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@allnes allnes Awaiting requested review from allnes allnes is a code owner

Assignees

No one assigned

Labels

task:all task:omp task:seq task:stl task:tbb task

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lisssq @codecov-commenter